Skip to content

Allow saving Parse Objects with Master Key #29

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Sep 16, 2014

Conversation

egreenmachine
Copy link
Contributor

Resolves #28.

Arguments are defaulted for backwards compatibility.

* @return null
*/
public function save()
public function save($useMasterKey=false)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a space before and after the equal signs in 3 locations, then I'll merge this. Thank you!

@egreenmachine
Copy link
Contributor Author

Added spaces to my branch

*
* @return null
*/
public static function saveAll($list)
public static function saveAll($list, $useMasterKey = false)
{
static::deepSave($list);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

woops, need to pass useMasterKey here too.

gfosco added a commit that referenced this pull request Sep 16, 2014
Allow saving Parse Objects with Master Key
@gfosco gfosco merged commit 99ec49b into parse-community:master Sep 16, 2014
@egreenmachine egreenmachine deleted the save-master-key branch September 16, 2014 22:27
@freakpol
Copy link

Note that code properly handles the case when there are more that one request, using thus the batch request. But in line 919 the $useMasterKey param should also be used to cover the case when there is only one request.

@egreenmachine
Copy link
Contributor Author

Thanks @freakpol, I submitted #32 this morning.

@freakpol
Copy link

Didn't noticed! Great work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add $useMasterKey parameter to ParseObject::save()
3 participants